-
Notifications
You must be signed in to change notification settings - Fork 334
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add flag to disable containers tooling (#1367) #1375
base: main
Are you sure you want to change the base?
Conversation
Nice! Any chance to separate styling changes from the behavior change? |
Actually it's enforced by the pre-commit hook. I can try to make a first commit with only formatting, would it be okay? |
That would be perfect, thank you! |
14b40b5
to
dc299a3
Compare
done :) |
@blackheaven, are you sure it's not your editor's tooling doing the formatting? This doesn't look like |
dc299a3
to
7fa15b0
Compare
I think my language server formatted twice, anyway, it's fixed now. |
The main reason why I haven't merged this yet is because I'm not convinced what the API should look like. Maybe we should add something like |
@Mic92 Thank you so much! |
Should we check that for other parameters? (i.e. |
abbc375
to
5d8d3ab
Compare
No description provided.